Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Support Message Attributes #15

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

Conversation

ncallaway
Copy link

This change supports message attributes by pulling them out of the request headers and storing them in the message object.

The receive action will generate the XML for any attributes stored on the message body.

@PlugIN73
Copy link

@iain why this wasn't merged?

@elliotchance
Copy link

@PlugIN73 because the build is broken and there are merge conflicts must be fixed before any consideration of merging can happen.

@jonnyshaw89
Copy link

Has this change been abandoned?
Without this fix in, fake_sqs is not compatible with the AWS Java SDK

@ncallaway
Copy link
Author

@jonnyshaw89 When I made this change the main branch had a failing build and I hadn't checked in on this PR since then. I'll get everything fixed up and resolve merge conflicts in the next couple of days.

@elliotchance
Copy link

Reviewed 6 of 6 files at r1.
Review status: all files reviewed at latest revision, all discussions resolved, some commit checks failed.


Comments from the review on Reviewable.io

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants